Skip to content

update actions and use hashes instead of version tag#60

Merged
steffyP merged 3 commits intomainfrom
update-actions
Feb 5, 2026
Merged

update actions and use hashes instead of version tag#60
steffyP merged 3 commits intomainfrom
update-actions

Conversation

@steffyP
Copy link
Member

@steffyP steffyP commented Jan 27, 2026

Updating github actions to latest released version and use hash as reference (following up #48)

  • updates actions/download-artifact@v4 to v7
  • updates actions/upload-artifact@v4 to v6
  • updates dawidd6/action-download-artifact@v6 to v12

Changed to hash for current major version only:

  • actions-cool/maintain-one-comment (still v3)

Additionally:

  • added on-push to run the CI tests for all pushes on main

@alexrashed
Copy link
Member

@steffyP This PR is actually already looking good, is this still missing anything?

@steffyP
Copy link
Member Author

steffyP commented Feb 4, 2026

@steffyP This PR is actually already looking good, is this still missing anything?

@alexrashed Ideally I would like to run the ephemeral workflow with the updated actions before merging, to ensure everything still works as expected. But that means we need to merge #61 first, and we are still waiting for a review.

@github-actions
Copy link

github-actions bot commented Feb 4, 2026

The ephemeral instance for the application preview has been shut down

@steffyP steffyP marked this pull request as ready for review February 4, 2026 17:18
@steffyP steffyP requested a review from alexrashed February 4, 2026 17:18
@steffyP
Copy link
Member Author

steffyP commented Feb 4, 2026

@alexrashed should we do a patch release after merging this?
then I would also add another commit here with version-update for the README code samples?

Copy link
Member

@alexrashed alexrashed left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome! Nice final cleanup step after a set of nice cleanups and refactorings! 🤩
If this is the last PR for the current "wave", then a release would be great!

@steffyP steffyP merged commit 1feaef8 into main Feb 5, 2026
6 checks passed
@steffyP steffyP deleted the update-actions branch February 5, 2026 08:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants